fix(files_external): skip BSD ftpd total N listing headers - #63197
fix(files_external): skip BSD ftpd total N listing headers#63197xusnitdinov wants to merge 1 commit into
Conversation
BSD ftpd prepends directory listings with a `total N` line. The raw LIST parser treated that as a file entry and threw, so external FTP storage against FreeBSD could not be browsed. Skip empty lines and `total` headers the same way flysystem does, while still parsing `.` / `..` as cdir/pdir. Fixes: nextcloud#37045 Signed-off-by: Xusnitdinov Azizbek <252487890+xusnitdinov@users.noreply.github.com>
|
Hello there, We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process. Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6 Thank you for contributing to Nextcloud and we hope to hear from you soon! (If you believe you should not receive this message, you can add yourself to the blocklist.) |
joshtrichards
added
bug
3. to review
2 participants
Summary
LISToutput with atotal Nline. The raw-list fallback inFtpConnectiontried to parse that as a file entry and threwMetadata can't be parsed from item 'total 17'.totalheaders the same way flysystem does../..are still parsed ascdir/pdir.Fixes #37045
Checklist
Made with Cursor